Merge branch 'wip/baedert/filechooser2' into 'master'

Assorted filechooser improvements

See merge request GNOME/gtk!783

Merge branch 'file-chooser-escape' into 'master'

file chooser: Prevent accidental search

Closes #1850

See merge request GNOME/gtk!784

file chooser: Prevent accidental search

When hitting Escape, the file chooser will go
into search mode, because the search entry
consumes the key to emit the ::search-stopped
signal. Recognize this situation and avoid
switching to search mode in this case.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1850

filechooserwidget: Use a center box for the search entry

This way we can avoid a slight position offset when we show or hide the
spinner during a search.

filechooserwidget: Allow keynav from the treeview to the search entry

window: Check whether the new focus widget is the old one

We generally do that for all properties.

treeview: Don't always grab_focus() when moving the cursor

This doesn't really make sense, we can assume that the treeview already
has the focus when it receives key events.

filechooserwidget: Remove unneeded function prototype

filechooserwidget: Don't focus treeview when searching

It's just weird that the search entry does not have focus while we type.

filechooserwidget: Remove some GdkEvent usage

filechooserbutton: Remove some dead code

colorbutton: Remove unneeded snapshot implementation

Adwaita: Bring filechooserbutton spacing back

Add some space between the icons and the text, like there was in gtk3.

entry: Fix primary icon allocation in RTL entries

The primary icon is on the right in RTL, so don't offset the text child
to the right for it.
Fixes the text and icon overlapping in the second entry
in tests/testentryicons

gl renderer: Add simple blend node implementation

treeview: Always update expand when allocating columns

It doesn't make sense to keep track of all the last_* values anymore now
that widgets only get allocated when their size changes anyway.
Remove all the associated (and thus now unused) flags as well.

treeview: Remove a useless check

treeview: Remove unused member

Merge branch 'popover-demos' into 'master'

Popover demos

See merge request GNOME/gtk!782

widget-factory: Make a resizing popover

Add an expander to a popover to test how
changing sizes works with popovers.

Merge branch 'file-chooser-search' into 'master'

file chooser: Fix Ctrl-L

Closes #1829

See merge request GNOME/gtk!781

widget-factory: Add some nested popovers

These are mainly for stress-testing our
popover implementations.

widget-factory: Add non-modal popovers

This makes is easy to test moving the toplevel
or repositioning the parent while the popover
is shown.

file chooser: Fix Ctrl-L

We were forwarding key events to the search entry
and unconditionally considered search started
afterwards. That is not correct, since things
like a Ctrl key press should not trigger search.
Fix this by only switching to search mode when
the event was actually consumed.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1829

Merge branch 'font-chooser-search' into 'master'

fontchooser: Fix type-to-search

Closes #1842

See merge request GNOME/gtk!780

fontchooser: Fix type-to-search

The key capture was interfering with other
entries in the dialog, so be smarter about
when we want to capture keys and when we
don't.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1842

searchentry: Always measure and allocate icon

Just measuring it (so the warning goes away) but then not using the
values will later underallocate the text widget.
Instead, always reserve space for the icon (which will inevitable be
visible as soon as the searchentry is actually being used).

Fixes #1831

Adwaita: separate '.view text' from textview>text

The first rule breaks GtkText inside GtkSearchEntry inside something
that has .view applied, as found in the file chooser when searching.

testsuite: Rewrite text diff to use GSubprocess

... instead of g_spawn(). Avoids having to create a temp file, too.

gl renderer: bind the texture framebuffer before rendering

When rendering to a texture, collecting the render ops might bind a
different framebuffer, so bind the one we want again before doing the
actual rendering.

tooltip: Initialize tooltip coords to given event coords

Otherwise the coordinates passed to the query-tooltip signal are always
0.

Merge branch 'wip/surface-transform-data' into 'master'

Surface transform listener fixes

See merge request GNOME/gtk!776

Merge branch 'fix-gdkwin32-move-drag' into 'master'

gdksurface-win32.c: Acquire root coordinates on drag/move

See merge request GNOME/gtk!693

widget: Move surface relative transform fields into its own struct

The purpose being to reduce the size of GtkWidgetPrivate. What is left
is a pointer to the allocated struct.

Updated Spanish translation

Merge branch 'wip/alexl/gl-debug' into 'master'

Add GL debugging integration

See merge request GNOME/gtk!771

widget: Remove leftover parent-changed handler removal

It was never added, as the listener was replaced by explicit code in
gtk_widget_(un)root(), but the removal code was left in place in error.

gdkglcontext: Only emit opengl debug calls if GDK_DEBUG=gl-debug

This avoids a potential performance cost in the non-debug case.

GskGLRenderer: Add debug groups

This adds debug groups in various places, including the debug
nodes if those are in use. This makes the traces in tools like
renderdoc much easier to read.

GskGLRenderer: Label various gl objects

This means the names are more useful in tools like renderdoc.

GdkGLContext: Add internal functions for KHR_DEBUG calls

This includes pushing and poping debug group messages and labeling
objects.

Merge branch 'parent-surface' into 'master'

Drop the parent surface getter and setter

See merge request GNOME/gtk!774

widget: Drop gtk_widget_set_parent_surface

And the getter, too. Widgets no longer have
extra surfaces that could serve as parent surfaces.

popover: Stop using gtk_widget_get_parent_surface

We can just use the surface of the parent.

tests: Stop using gtk_widget_get_parent_surface

We can just use the surface of the parent.

a11y: Stop using gtk_widget_get_parent_surface

We can just use the surface of the parent.

Merge branch 'wip/widget-position-changed-callbacks' into 'master'

widget: Add surface relative position changed callback

See merge request GNOME/gtk!760

Merge branch 'fix-and-improve-linked-button-style' into 'master'

themes: Fix and improve linked button style

Closes #1294

See merge request GNOME/gtk!730

Merge branch 'wip/jimmac/type-experiments' into 'master'

Adwaita: generic type styles

See merge request GNOME/gtk!772

Adwaita: generic type styles

- try to define a basic typographic structure

See https://gitlab.gnome.org/GNOME/gtk/issues/1808

Merge branch 'csserror-g-decls' into 'master'

csserror: Add G_BEGIN_DECLS and G_END_DECLS

See merge request GNOME/gtk!773

gl renderer: Keep op builder around

GL keeps the unoform state per-program, but not per-frame. So, we can't
pretend that this works for us. Keep the RenderOpBuilder around for the
entire lifetime of the renderer instead.

csserror: Add G_BEGIN_DECLS and G_END_DECLS

Necessary when the header file is included in C++ code.

Merge request !773

widget: Add surface relative transform changed callback

Added two new private GtkWidget API:

 * gtk_widget_add_surface_transform_changed_callback()
 * gtk_widget_remove_surface_transform_changed_callback()

The intention is to let the user know when a widget transform relative
to the surface changes. It works by calculating the surface relative
transform during allocation, and notifying the callbacks if it changed
since last time. Each widget adds itself as a listener to its parent
widget, thus will be triggered if a parents surface relative transform
changes.

gl renderer: Sync gpu and cpu default alpha value

Otherwise we might end up not passing the new value to the GPU.

gl renderer: Properly apply opacity in cross-fade shader

Updated Spanish translation

Merge branch 'simplify-fixes' into 'master'

Simplify fixes

See merge request GNOME/gtk!770

builder-tool: Check canonical names

Always canonicalize names before comparing.
We were missing properties like left_attach,
since we were comparing them to left-attach.

builder-tool: Fix property check

Embarrassingly, there was a missing else
causing us to check all packing properties
as object properties, never finding them.

eventcontrollermotion: Document missing signal parameters

cssprovider: Fix a parameter name mismatch

testsuite: Don't pretend we're a GTest

This is a meson test, not a GTest thing. So:

- Use g_print(), not g_test_message
That makes meson test --verbose print the actual log messsages.

- Don't g_assert() all the time
Instead, run tests through to the end and just return a non-0 exit
status.

gl renderer: begin/end_frame around do_render

This fixes rendering to a texture on intel hardware. The glClear calls
would throw a GL_FRAMEBUFFER_INCOMPLETE error here, because the
gsk_gl_driver_begin_frame() call in do_render() reset the framebuffer
object in use.

GdkWaylandDevice: Don't recreate the default cursor every frame

Save the information whether the cursor in use is the default one, and
don't create a new cursor object in that case.
We previously created a new cursor object every frame just to compare it
to the current cursor in use and then throw it away.

Updated Spanish translation

Updated Spanish translation

Merge branch 'forgotten-file' into 'master'

window: Remove dropped api from headers

See merge request GNOME/gtk!769

window: Remove dropped api from headers

This was forgotten in fed2db1493606342.

gdk: Remove gdk_surface_invalidate_rect() from public API

It's still available for backends, but public API (read: GTK) is
meant to only call gdk_surface_queue_expose() and track its
invalid region itself.

Merge branch 'unused-typehint' into 'master'

gdk: Remove an unused field

See merge request GNOME/gtk!767

gdk: Remove an unused field

Its not used, so get rid of it.

Merge branch 'drop-skip-pager' into 'master'

window: Drop some x11-specific apis

See merge request GNOME/gtk!766

surface: Drop group api

This was only ever implemented on X11, and
GTK is not using it at all. Relegate it to
x11-specific api.

window: Drop some x11-specific apis

The skip-taskbar, skip-pager and urgency hints were
only ever implemented for X11, and are not very useful
with modern desktops. Relegate the functionality to
x11 backend api, and drop the GtkWindow api.

Fix a compiler warning

wayland: Improve debug output for events

We were missing surfaces in a few places.

Merge branch 'cairo-borders-fixage' into 'master'

Cairo borders fixage

See merge request GNOME/gtk!765

rendernode: Fix border rendering

This fixed the reftest introduced in the previous commit.

I'm using a mesh gradient here instead of drawing 4 individual sides to
avoid artifacts when those sides overlap in rounded corners.

reftests: Add a reftest for rendering failure

c18d2872f292607c921c959e6e97c4d6635bdfd2 introduced a misrendering in
the Cairo fallback rendering code while fixing another one.

This test exposes that problem.

Related: !756

window: Fix an oversight

We must still reset cursors.

Merge branch 'cursor-things' into 'master'

Cursor things

See merge request GNOME/gtk!764

window: Stop juggling cursors

We have convenience api for this.

text: Stop juggling cursors

We have a convenience api for this.

label: Stop juggling cursors

We have a convenience api for this.

Merge branch 'style-prop-change' into 'master'

Style prop change

See merge request GNOME/gtk!763

css: Kerning affects text size

Thats the point. And we were missing the proper
flags to make GTK do the right thing when
changing font-kerning in the inspector.

css: Some updates to internal docs

Minor corrections to the AFFECTS flags docs.

Merge branch 'letterspacing-resize' into 'master'

css: Letterspacing affects text size

See merge request GNOME/gtk!762

Merge branch 'wip/carlosg/tablet-cursors-hidpi-master' into 'master'

gdk/wayland: Listen to wl_surface events on tablet cursors

Closes #1675

See merge request GNOME/gtk!761

css: Letterspacing affects text size

It does. And we were missing the proper
flags to make GTK do the right thing when
changing letter-spacing in the inspector.

gdk/wayland: Listen to wl_surface events on tablet cursors

And update the surface accordingly (eg. scale on hidpi). The mechanism
that did that for wl_pointer has been made generic so it can be shared
with tablets too.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1675

Merge branch 'handle-tool-scale-fix' into 'master'

gdkdevice-wayland: Update pointer scale on tablet tool proximity events

See merge request GNOME/gtk!582

Merge branch 'wayland-cursor-hotspot' into 'master'

wayland: Avoid using uninitialized memory

Closes #1328

See merge request GNOME/gtk!757

wayland: Avoid using uninitialized memory

_gdk_wayland_cursor_get_buffer was not initializing
its out variables in the 'not found' case. This
was showing up in protocol traces as garbage hotspots
being sent to the compositor.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1328

testsuite: Add theme-validate test

The test just loads all of GTK's themes and makes sure they cause no
errors or warnings from the CSS parser.

themes: Regenerate CSS

Seems to have been forgotten a few times.

cssprovider: Export correct API

gtk_css_provider_get_named() is the old GTK3 style API to load themes.

Instead, export the function we currently use,
gtk_css_provider_load_named().

As a side effect we allow people to load a theme as often as they want
without conflicting with GTK's theme.

Survive absence of librsvg without criticals

We can't guarantee that we can load svgs, so
we shouldn't spew criticals when some of our
own resources fail to load due to that reason.

Avoid a crash without librsvg

gtk_css_image_recolor_load_texture may leave
recolor->text as NULL without setting error.
No need to crash for that.